Remove dead upgrades through v7#101
Merged
algoradam merged 7 commits intoalgorand:masterfrom Jun 26, 2019
Merged
Conversation
Remove backward-compatibility code for old-style ephemeral keys. We switched to fine-grained ephemeral keys in consensus v3.
In particular, always support closeout transactions
In particular, always support explicit ephemeral-key parameters
In particular, always use the new "twin seeds" seed algorithm
derbear
reviewed
Jun 26, 2019
derbear
reviewed
Jun 26, 2019
| // PKSigOld is unused; however, unfortunately we forgot to mark it | ||
| // `codec:omitempty` and so it appears (with zero value) in certs. | ||
| // This means we can't delete the field without breaking catchup. | ||
| PKSigOld ed25519Signature `codec:"ps"` |
Contributor
There was a problem hiding this comment.
@zeldovich Out of scope for this PR, but I was thinking of one way we can deprecate this field: Since signatures are never committed to in any way (they're part of the cert but they themselves are not signed), we could update all archival servers to supply certificates with this field missing. After all archives have updated, it should be safe to drop the field.
derbear
approved these changes
Jun 26, 2019
Closed
tmc
pushed a commit
to tmc/go-algorand
that referenced
this pull request
Mar 7, 2025
* Remove IncorrectBalLookback (fixed with v5 upgrade) * Remove support for coarse-grained ephemeral keys Remove backward-compatibility code for old-style ephemeral keys. We switched to fine-grained ephemeral keys in consensus v3. * Remove pre-consensus-v4 backward compatibility code In particular, always support closeout transactions * Remove pre-v6 backward compatibility code In particular, always support explicit ephemeral-key parameters * Remove pre-v8 backward compatibility code In particular, always use the new "twin seeds" seed algorithm * Further remove support for old-style ephemeral sigs * Deprecate all consensus versions before v7 and fix tests
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove backward-compatibility code for some of the upgrades that happened before Mainnet launch and thus can be removed without impacting catchup. This PR should be reviewed one commit at a time; each commit removes backward compatibility code for one protocol upgrade so will be easier to read in isolation.
This PR removes upgrades through (but not including) v7. Remaining dead upgrades will be removed in a future PR.